Draft
Conversation
…ntime env url for internal docker communication
Alameen688
commented
Oct 4, 2025
Comment on lines
-22
to
-31
| op.create_table('chat', | ||
| sa.Column('message', sqlmodel.sql.sqltypes.AutoString(length=1024), nullable=True), | ||
| sa.Column('is_system', sa.Boolean(), nullable=False), | ||
| sa.Column('created_at', sa.DateTime(timezone=True), server_default=sa.text('now()'), nullable=False), | ||
| sa.Column('updated_at', sa.DateTime(), nullable=False), | ||
| sa.Column('id', sa.Uuid(), nullable=False), | ||
| sa.Column('course_id', sa.Uuid(), nullable=False), | ||
| sa.ForeignKeyConstraint(['course_id'], ['course.id'], ondelete='CASCADE'), | ||
| sa.PrimaryKeyConstraint('id') | ||
| ) |
Collaborator
Author
There was a problem hiding this comment.
I could revert this, but I needed it to get the fresh migrations to run on my end without issues with duplicate create chat table.
Collaborator
|
@Alameen688 see this pr too, so that effort is not duplicated #33 |
michaelgichia
requested changes
Oct 6, 2025
Collaborator
michaelgichia
left a comment
michaelgichia
left a comment
There was a problem hiding this comment.
Reposting for visibility:
I really like your changes, but maybe we could add more as @ayodeji.moronkeji has mentioned.
- Futuristic landing page
- Mobile responsiveness
- Style the components inside components/ui so that the changes are applied throughout the page.
- Share screenshots of dark/light mode.
- Don't remove the theme provider
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Overview
Implemented design system based on Miro screenshot across authentication pages and dashboard pages
Key Changes
Sections affected
sb-*system)Layout and colors
Three-column layout: Left sidebar, main content, right sidebar (Project Settings)
Fixed elements: Header and right sidebar now sticky/fixed
Tab navigation: Consistent styling across course pages and quiz sessions
Color scheme:
#101827(dark background)#182130(content background)#1f2937(surface background)AppSidebar: Dynamic course list, fixed "New Project" button
ProjectSettings: Empty state with gear icon, document upload
Quiz Session: Integrated tab navigation matching course page layout
User Account: Moved to top header with dropdown menu
Theme management: Removed light/dark toggle, enforced single dark theme per Miro UI
Layout consistency: Fixed positioning for headers, sidebars, and tabs
Component reuse: Leveraged existing components for new layouts
Screenshots
Authentication Pages
Dashboard & Course Management